-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: enable prefetch dependencies in Konflux #1323
Conversation
54355c1
to
312bbca
Compare
4db86fa
to
cc7e054
Compare
/retest |
1 similar comment
/retest |
cc7e054
to
34be2bd
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #1323 +/- ##
==========================================
- Coverage 36.81% 36.50% -0.31%
==========================================
Files 158 159 +1
Lines 2548 2586 +38
Branches 599 605 +6
==========================================
+ Hits 938 944 +6
- Misses 1428 1460 +32
Partials 182 182 ☔ View full report in Codecov by Sentry. |
@ernesgonzalez33 hi, #1322 merged upstream |
I'll rebase and let you know! |
Signed-off-by: Ernesto González <ergonzal@redhat.com>
Signed-off-by: Ernesto González <ergonzal@redhat.com>
34be2bd
to
bbcf523
Compare
Quality Gate passedIssues Measures |
/retest |
@yaacov @mnecas I know that the test is failing, but this is due to an architectural decision on Konflux to test components inside an app together. So it is testing this specific PR at the same time as the main branch of the must-gather repo (which was failing), so until we don't merge these two working, it will keep failing. So my suggestion is to merge this PR and then test it with the #1329 |
Needs #1322 merged before